Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CR] Possibility to fix experimental cyborgs at Rubic's Castle #74054

Conversation

Vgoloshivskiy
Copy link
Contributor

Added possibility to fix cyborgs at Rubic's

Summary

Content "Added a way to fix cyborgs at rubic's castle"

Purpose of change

Add easier way to fix cyborgs,
I could do it

In future hopefully before stable 0.H it would be possible to add mission to rescue desparete Hub-01 mercenary who got themselves a bionic through Doc or autodoc and ended up as prototype Cyborg (for example mercenary had alergies to a lot of food or had astma).
This is a way to help them but also push a bit conflict with exodii a bit closer.

Describe the solution

Added dialogs and a bit of other content in order for this to work
spawned npc should be of hub-01 mrecenary faction

Describe alternatives you've considered

This is new content so i didn't think about them

Testing

Run in dev mode to Rubic with ton of items for selling and inactive bodies and tried saving them -> NPC spawned -> but they probably gonna need some new dialog options;

Additional context

I'm not a native english speaker so i have no idea is my grammar correct or no I will be checking it right now.
It also need translation in Rubic's language cause i have no idea how to make it passable for language

Added possibility to fix cyborgs at Rubic's
@I-am-Erk
Copy link
Member

This is one I should take a look at personally. I'm pretty busy but I'll try to get on it over the next few days. I'm moving it to draft so that I can do that. Looks exciting though!

@I-am-Erk I-am-Erk marked this pull request as draft May 24, 2024 19:51
@Vgoloshivskiy
Copy link
Contributor Author

Vgoloshivskiy commented May 24, 2024

Do what you must, after my pull requests will be merged i will try to create mission.

@github-actions github-actions bot added NPC / Factions NPCs, AI, Speech, Factions, Ownership [JSON] Changes (can be) made in JSON labels May 24, 2024
data/json/npcs/classes.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
Vgoloshivskiy and others added 3 commits May 24, 2024 23:19
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label May 24, 2024
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label May 24, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label May 24, 2024
@github-actions github-actions bot removed the json-styled JSON lint passed, label assigned by github actions label May 25, 2024
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label May 25, 2024
@Vgoloshivskiy
Copy link
Contributor Author

I think crucial bit is to add mission to return them to the hub so they can pay you from their stash (payment should be oblivious to the player and can be everything from hub coins and nanofab templates to somesing exotic like artifacts or location of undiscovered dungeon)

Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@github-actions github-actions bot added the stale Closed for lack of activity, but still valid. label Jun 24, 2024
@RenechCDDA
Copy link
Member

I know Erk isn't going to have any time in the near future, so I'm going to go ahead and put this back to ready for review. Sorry about the holdup.

@RenechCDDA RenechCDDA marked this pull request as ready for review June 24, 2024 20:03
@github-actions github-actions bot removed the stale Closed for lack of activity, but still valid. label Jun 24, 2024
@github-actions github-actions bot removed json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jun 28, 2024
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jun 28, 2024
@Vgoloshivskiy
Copy link
Contributor Author

There are some failing checks but i m honestly not sure are they related to me or not. I used ChatGpt For Rubic translated text as writing dialogue in non-existing language is something that i m too afraid to do

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jul 21, 2024
Copy link
Member

@Maleclypse Maleclypse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CBMs I cut are all uniques that should only be spawning nether labyrinths. I also changed faction to your followers after surgery. This is looking fairly good to go though once those changes commit.

data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_itemlist.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
data/json/npcs/exodii/exodii_merchant_talk.json Outdated Show resolved Hide resolved
Copy link
Member

@kevingranade kevingranade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used ChatGpt For Rubic translated text as writing dialogue in non-existing language is something that i m too afraid to do

Please pull the ChatGPT produced content back out. ChatGPT is a copyright infringing tool we want nothing to do with.
By submitting changes you are asserting that you wrote it, if you can't make that assertion, don't submit it.

@Zireael07
Copy link
Contributor

To add what Kevin said, it's also highly likely that ChatGPT gave you a mix of existing languages instead of what you wanted.

@Vgoloshivskiy
Copy link
Contributor Author

I used ChatGpt For Rubic translated text as writing dialogue in non-existing language is something that i m too afraid to do

Please pull the ChatGPT produced content back out. ChatGPT is a copyright infringing tool we want nothing to do with. By submitting changes you are asserting that you wrote it, if you can't make that assertion, don't submit it.

Will do tommorow can't right now

@Vgoloshivskiy
Copy link
Contributor Author

I used ChatGpt For Rubic translated text as writing dialogue in non-existing language is something that i m too afraid to do

Please pull the ChatGPT produced content back out. ChatGPT is a copyright infringing tool we want nothing to do with. By submitting changes you are asserting that you wrote it, if you can't make that assertion, don't submit it.

Will do tommorow can't right now

Done

Copy link
Contributor

Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details.

Click to expand
  • Fer this 'un, I can yank out their bits 'n give 'em to ya, or I can jus' take out the busted pieces so they can still use th' rest o' stuff in 'em. Elsewise, why don't ya jest give 'im to me? I reckon I can turn 'im into a right useful worker.
  • Gimme spek, 'n fetch 'em 'ere.
  • Nah, mate, still operatin
  • Rubic leads a person to you.
  • This 'unwon't be twitching', too much grey lost to dead. But ye can salvage bits from 'em or born 'em for a walker, or just cut out the rot 'n grey link so ya can miss 'em proper.
  • [Broken Cyborg] Can you save my buddy who looks a bit rough around the edges, but they still haven't flatlined?

This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to tools/spell_checker/dictionary.txt so they will not trigger an alert next time.

Hints for adding a new word to the dictionary
  • If the word is normally in all lowercase, such as the noun word or the verb does, add it in its lower-case form; if the word is a proper noun, such as the surname George, add it in its initial-caps form; if the word is an acronym or has special letter case, such as the acronym CDDA or the unit mW, add it by preserving the case of all the letters. A word in the dictionary will also match its initial-caps form (if the word is in all lowercase) and all-uppercase form, so a word should be added to the dictionary in its normal letter case even if used in a different letter case in a sentence.
  • For a word to be added to the dictionary, it should either be a real, properly-spelled modern American English word, a foreign loan word (including romanized foreign names), or a foreign or made-up word that is used consistently and commonly enough in the game. Intentional misspelling (including eye dialect) of a word should not be added unless it has become a common terminology in the game, because while someone may have a legitimate use for it, another person may spell it that way accidentally.

Audit CBMs in prototype cyborg recovery.
@github-actions github-actions bot removed json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 25, 2024
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 26, 2024
@github-actions github-actions bot added BasicBuildPassed This PR builds correctly, label assigned by github actions and removed BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 13, 2024
@Maleclypse Maleclypse merged commit b84853d into CleverRaven:master Aug 17, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants